Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix lecturer header card: add subjects bar #40

Merged
merged 11 commits into from
Nov 29, 2024

Conversation

Dmatrushka19
Copy link
Contributor

@Dmatrushka19 Dmatrushka19 commented Nov 25, 2024

Изменения

Добавил статус-бар с предметами в карточку преподавателя

Детали реализации

-Изменил три файла: LectureHeaderCard.vue, LecturePage.vue, ReviewPage.vue<
-В LectureHeaderCard.vue добавил наследование из родительского файла объекта subjectsName
-В LecturePage.vue и ReviewPage.vue добавил subjects в вызов дочернего файла
-Убрал вывод "ошибка", так как выводилось некорректно

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@Dmatrushka19
Copy link
Contributor Author

Dmatrushka19 commented Nov 25, 2024

Есть проблема с форматированием (в двух файлах идентичнаяLecturePage.vue и ReviewPage.vue):
image
При большом количестве предметов появляются функциональные стрелки перемотки, но для полной красоты и симметрии надо бы сделать целочисленное проматывание предметов, чтобы не было предмета, отображающегося только на половину
Снимок экрана 2024-11-26 010414
Не знаю, что будет в мобильной версии, так как if mobile не трогал

src/components/LecturerHeaderCard.vue Outdated Show resolved Hide resolved
vite.config.ts.timestamp-1732570504023-45fb8acbd1d38.mjs Outdated Show resolved Hide resolved
src/pages/ReviewPage.vue Outdated Show resolved Hide resolved
src/pages/ReviewPage.vue Outdated Show resolved Hide resolved
src/components/LecturerHeaderCard.vue Outdated Show resolved Hide resolved
@Dmatrushka19 Dmatrushka19 merged commit fba74f7 into main Nov 29, 2024
2 checks passed
@Dmatrushka19 Dmatrushka19 deleted the fix-lecturer-headerCard-add-subjects branch November 29, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Добавить отображение предметов на страницу преподавателя
2 participants